-
Notifications
You must be signed in to change notification settings - Fork 11
🤖 Add release workflow for automated binary publishing #168
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
- Create release.yml workflow triggered on release publish - Configure electron-builder to publish to GitHub releases - Add workflow documentation explaining release process - Keep build steps in sync with build.yml for consistency Release process: 1. Bump version in package.json 2. Create draft release with notes on GitHub 3. Publish release → workflow builds and attaches binaries 4. Users download from GitHub releases The workflow reuses the same build setup as build.yml but publishes artifacts to GitHub releases instead of workflow artifacts.
- Create composite action .github/actions/setup-cmux for common setup - Reduces 4 steps (checkout, bun, install, build) to 1 step - Extract macOS code signing to scripts/setup-macos-signing.sh - Reduces ~17 lines of inline bash to 1 step - Makes signing logic testable locally - Update build.yml to use composite action and signing script - Update release.yml to use composite action and signing script Benefits: - Single source of truth for build setup - Changes to build process update both workflows - Workflows are more readable (fewer lines) - Code signing script can be tested independently
Add contents: write permission so electron-builder can upload binaries to GitHub releases. Without this, the default token only has read access and uploads fail with 403.
c4570b6 to
e49d451
Compare
ImageMagick must be installed before building (for icon generation), so the build step needs to happen after platform-specific setup.
|
@codex review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Icons are generated during build, so ImageMagick must be installed before building in both CI and release workflows.
|
@codex review |
Summary
Adds automated release workflow that builds and publishes macOS and Linux binaries when a GitHub release is published.
Release Process
Changes
release: published--publish alwaysto attach artifacts to GitHub releaseDRY Approach
The release workflow reuses the same build setup as :
Key difference: uses for PR verification, while uses with to attach binaries to releases.
Testing
This can be tested by:
Generated with
cmux